Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add DPI awareness to manifest (fix #7) #99

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MattSturgeon
Copy link

I tested @roblarky's suggestion and found it fixes the DPI issues completely for me. I've marked him as the author since all changes are his, but it would be good to get this into an AltDrag release in the near future 😃


Testing

Here's a .zip of my build directory in case anyone wants to try out these changes without having to compile them.

After extracting, you can either run AltDrag.exe directly from the bin directory. You can also run the installer (AltDrag-1.1.exe), but be sure to manually undo the workaround as it will conflict/override these changes.

Alternatively you can of course use the normal method to clone and build this branch if you don't want to trust my binaries.

@roblarky
Copy link

roblarky commented Jan 2, 2019

Glad to hear!

@Arnaudv6
Copy link

Arnaudv6 commented Feb 7, 2019

Hello and thanks for this awesome program,
Altdrag is the first thing I install on windows when forced to use this OS. (at work for instance)
Much appreciated is the portable app: so no admin right is needed on locked machines.

This fix calls for a release I deem.
A release is needed urgently: last release dates three and a half years back.
I can fill a separate bug if needed.

@yatli
Copy link

yatli commented Jul 19, 2019

Reporting in: it totally fixed the problem, thanks!!!

In particular I should mention: it fixed the problem when dragging across two monitors with different DPI, there was a problematic feedback loop, which turns the target window infinitely bigger and bigger.

Now it looks like the coordinates are consistent across the boundaries!

@bufferUnderrun
Copy link

bufferUnderrun commented Mar 1, 2021

Is there anyone who can fork this project and apply criticals PR like this one ?
...And provide .exe ?

Thanks

@Arnaudv6
Copy link

Arnaudv6 commented Mar 1, 2021

I just tried alternative mentioned here #124, seems more up-to-date

@bufferUnderrun
Copy link

@Arnaudv6 Thanks, Absolutely Fabulous, v1.4 working great !!

This excellent fork should have a better visibility, why not adding a redirect to ?

@delchiaro-bh
Copy link

This solved all the issues for dual screen and different DPI settings in each screen on windows 10!
Super smooth!

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants